-
Notifications
You must be signed in to change notification settings - Fork 1k
fix: arduino-cli upload failed (version started from 0.19.0 to 0.21.0) #1654
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Error during Upload: Property 'upload.tool.serial' is undefined Issue present for arduino-cli version less than 0.21.0 See arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
I'm not entirely sure this fix is correct. I have not tested, but AFAUI this will no longer allow changing the upload method using the "menu" entries using older arduino-cli, since you now set a new-style upload tool for the board as a whole, but no (new-style) tool for the menu entries (only old-style tool lines, but those are not converted in 0.21). IOW, I think you need lines like below (instead of the ones you added now):
Also, might be good to mention the full range of affected arduino-cli versions, looks like this problem was introduced in 0.19 with pluggable discovery, it was already working before that. |
In fact with IDE it works but not with the cli.
Unfortunately this is not recognized properly. I guess no fix can be provided for this. Now I wondering if we could use the pluggable discovery feature.... |
This reverts partially commit a4e4377. Currently, defining a default upload tools prevents flashing with arduino-cli as default methods is always used even if new one is passed as fqbn parameters. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
This reverts partially commit a4e4377. Currently, defining a default upload tools prevents flashing with arduino-cli as default methods is always used even if new one is passed as fqbn parameters. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
new version 1.2.1 raised an error about default upload method. But it could not be fixed easily as Arduino seems not manage submenu level for upload method. See: stm32duino#1654 arduino/arduino-cli#1444 Signed-off-by: Frederic Pillon <[email protected]>
Error during Upload: Property 'upload.tool.serial' is undefined
Issue present for arduino-cli version started from 0.19.0 to 0.21.0.
See arduino/arduino-cli#1444
/cc @matthijskooijman